Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improved detection of line endings in ChunkBuffer #42

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

switchCTRL
Copy link

Added carriage return + line feed (\n\r) as possible line endings to 'readLine()'.

Otherwise if the server delivers a response like:
data: {...}\n\r data: {}
the response cannot be parsed correctly in 'ServerSentEventsTransport.open()'

Added carriage return + line feed (\n\r) as possible line ending to 'readLine()'
@clayellis
Copy link

@jordancamara can you take a look at this?

@clayellis clayellis mentioned this pull request Jun 7, 2019
@switchCTRL
Copy link
Author

switchCTRL commented Jun 8, 2019

Is there someone with write access? If so i will update the PR.

@clayellis
Copy link

It seems like everyone with write access have gone silent. I'd love to see this merged. It solved an issue for me.

@kamrankhan-chilindo
Copy link
Contributor

Seems like I don't have the rights to merge too, I would like someone to update the pods too, to fix some bugs Starscream. May be forking is the only way for now

@kamrankhan07
Copy link
Contributor

I have updated this and the other pull request into my fork, and updated the pods too. I didn't test much, though everything seems to work fine. Not much changes in the code, however I will be testing all the new changes this week. If anyone finds an issue then please let me know.
https://github.com/kamrankhan07/SignalR-Swift

aclima93 added a commit to aclima93/SignalR-Swift that referenced this pull request Apr 28, 2020
@aclima93
Copy link

aclima93 commented Apr 28, 2020

Shouldn't it be \r\n instead? Great find nonetheless!

@santhoshappdev
Copy link

still issue is there.
#44 (comment)

@santhoshappdev
Copy link

Objective C code working in swift (using bridge header) - No reconnection issues and other. Dear @auther please check on swift version. Thank for your help

https://github.com/DyKnow/SignalR-ObjC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants